Skip to content

[docs] Add docs for env.* JVM and process options#3618

Merged
luoyuxia merged 1 commit into
apache:mainfrom
orangeCatDeveloper:docs-env-vars
Jul 18, 2026
Merged

[docs] Add docs for env.* JVM and process options#3618
luoyuxia merged 1 commit into
apache:mainfrom
orangeCatDeveloper:docs-env-vars

Conversation

@orangeCatDeveloper

Copy link
Copy Markdown
Contributor

Purpose

Linked issue: close #2513

The env.* options that the bin/ startup scripts read from conf/server.yaml were undocumented. This adds an Environment section to the Server Configuration page covering the JVM and process-launch options.

Brief change log

  • Add an ## Environment section to maintenance/configuration.md documenting the options that bin/config.sh reads and applies at process launch:
    • env.java.home
    • env.java.opts.all
    • env.java.opts.coordinator-server
    • env.java.opts.tablet-server
    • env.pid.dir
  • Cross-reference the Logging docs, which already document the env.log.* options, to avoid duplication.

Not included (from the issue's key list):

  • env.log.dir / env.log.level / env.log.max / env.stdout-err.redirect-to-file — already documented under Logging.
  • remote.data.dir — already documented in the same page's Common section.
  • env.ssh.opts and zookeeper.heap.mb — currently read into shell variables (FLUSS_SSH_OPTS, ZK_HEAP) but not wired to any behavior in the shipped scripts, so documenting them as functional options would be misleading. Happy to include them if they are intended to work.

Tests

Docs-only change. Built the site locally (build_versioned_docs.sh + docusaurus build); the new cross-reference link resolves and the change adds no broken links.

API and Format

No API or storage format changes.

Documentation

This PR is the documentation.

@orangeCatDeveloper
orangeCatDeveloper marked this pull request as draft July 8, 2026 07:37
@orangeCatDeveloper
orangeCatDeveloper marked this pull request as ready for review July 8, 2026 07:37
@luoyuxia
luoyuxia requested a review from Copilot July 18, 2026 03:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds missing documentation for env.* launch-environment options used by Fluss distribution startup scripts, improving operator discoverability on the Server Configuration page.

Changes:

  • Document JVM-selection and JVM-option keys (env.java.home, env.java.opts.*) and PID directory (env.pid.dir) in a new Environment section.
  • Add a cross-reference to existing Logging docs for env.log.* and env.stdout-err.redirect-to-file to avoid duplication.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread website/docs/maintenance/configuration.md Outdated
Comment thread website/docs/maintenance/configuration.md Outdated
@luoyuxia
luoyuxia merged commit cc93a86 into apache:main Jul 18, 2026
1 check passed
@orangeCatDeveloper
orangeCatDeveloper deleted the docs-env-vars branch July 18, 2026 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add docs for JAVA_OPTS related vars

3 participants